Skip to content

Codex/archive obsolete branches 20260709#449

Merged
BigSimmo merged 105 commits into
mainfrom
codex/archive-obsolete-branches-20260709
Jul 9, 2026
Merged

Codex/archive obsolete branches 20260709#449
BigSimmo merged 105 commits into
mainfrom
codex/archive-obsolete-branches-20260709

Conversation

@BigSimmo

@BigSimmo BigSimmo commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

Verification

  • npm run verify:cheap
  • npm run verify:ui when UI, routing, styling, browser behavior, reduced-motion, or forced-colors behavior changed
  • npm run verify:release before release or handoff confidence claims
  • npm run format:check
  • npm run eval:retrieval:quality (must stay 36/36) when retrieval, ranking, selection, chunking, or scoring behavior changed — CI cannot run it (needs live keys), so run it locally and paste the summary. A metadata/governance-weighting change once buried correct docs (recall 1.0→0.76) and only this eval caught it.
  • npm run eval:rag -- --limit 15 + npm run eval:quality -- --rag-only when answer generation, the synthesis prompt, or answer post-processing changed (grounded-supported must not drop; citation-failure 0)
  • npm run check:production-readiness when clinical workflow, privacy, environment, Supabase, source governance, or deployment behavior changed
  • npm run check:deployment-readiness when deployment startup, hosting, or rollout behavior changed

Clinical Governance Preflight

Complete this section when the change touches ingestion, answer generation, search/ranking, source rendering, document access, privacy, production env, or clinical output.

  • Source-backed claims still require linked source verification before clinical use
  • No patient-identifiable document workflow was introduced or expanded without explicit governance approval
  • Supabase target remains Clinical KB Database (sjrfecxgysukkwxsowpy)
  • Service-role keys and private document access remain server-only
  • Demo/synthetic content remains clearly separated from real clinical sources
  • Source metadata, review status, and outdated/unknown-source behavior remain conservative
  • Deployment classification/TGA SaMD impact was checked when clinical decision-support behavior changed

Notes

BigSimmo and others added 30 commits July 5, 2026 22:01
- Hide footer Evidence/Sources chips on phone hero composers; scope stays in + menu
- Suppress bottom-dock composer flash until hero portal slot is ready
- Increase composer action/send touch targets to 44px on phones
- Update Playwright tests for scope menu, Answer home geometry, and stress fallback
…rdening

fix(access): complete public retrieval scope and production access hardening
Replace the answer-mode composer banner and post-answer card with a shared
compact chip row that matches the footer search setup styling.

Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
…d modes

Unify empty-state quick actions with compact chips, move mobile follow-ups
into the footer composer, apply example chips to all search modes, add chip
truncation, update mockups/tests, and trim unused ClinicalDashboard imports.

Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Remove nested worktree references from the plugin branch and ignore
.worktrees/ so checkout and required status checks can pass.

Co-authored-by: Cursor <cursoragent@cursor.com>
cursoragent and others added 20 commits July 9, 2026 08:32
- Backfill NULL document_images.index_generation_id during re-stamp
- Count globally forced embedding eval cases in retrieval summaries

Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Require branch, HEAD, and scope to match before skipping cleanup reviews, and allow ledger appends as the sole edit during pure review runs.

Co-authored-by: Cursor <cursoragent@cursor.com>
Wrap issues.listComments and issues.createComment in explicit try/catch paths that log a clear notice and fail the job deterministically on auth, rate-limit, and client API errors while preserving skip cases.

Co-authored-by: Cursor <cursoragent@cursor.com>
Use SHA-scoped PR labels and concurrency to prevent duplicate resolve requests, skip non-actionable reviews, document the auto-resolve confirmation exception, and add CI workflow guard checks.

Co-authored-by: Cursor <cursoragent@cursor.com>
Document manual re-trigger, add resolve completion checklist, cover top-level PR comments in auto-resolve, and link the review protocol from README.

Co-authored-by: Cursor <cursoragent@cursor.com>
Queue PR/head workflow runs, create SHA labels before applying them, post the resolve comment before the label lock, and skip resolve completion summaries on issue_comment events.

Co-authored-by: Cursor <cursoragent@cursor.com>
Treat issues.get and listComments permission failures as warnings so the workflow stays green when the token cannot read PR metadata.

Co-authored-by: Cursor <cursoragent@cursor.com>
@supabase

supabase Bot commented Jul 9, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@BigSimmo
BigSimmo enabled auto-merge (squash) July 9, 2026 13:04
@gitguardian

gitguardian Bot commented Jul 9, 2026

Copy link
Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
34676489 Triggered Generic Password 2b70edc scripts/generate-drift-manifest.ts View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 94bd8771-0fef-4318-88cf-6dab27e02129

📥 Commits

Reviewing files that changed from the base of the PR and between 230ec3f and 98e4388.

📒 Files selected for processing (1)
  • BRANCH_ARCHIVE_20260709.md

📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Added a branch archive record documenting preserved remote branch history and the archive date.
    • Included a reference list of retained branch tips with their commit identifiers and messages for future lookup.

Walkthrough

A new markdown file, BRANCH_ARCHIVE_20260709.md, was added documenting an archive of obsolete remote branch tips. It describes the "ours" merge strategy used, records the archive date, and lists preserved remote branch references with their commit SHAs and messages.

Changes

Branch Archive Documentation

Layer / File(s) Summary
Archive document content
BRANCH_ARCHIVE_20260709.md
New file documenting the "ours" merge strategy, archive date (2026-07-09), and an enumerated list of preserved remote branch tips with commit SHAs and messages.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 11
✅ Passed checks (11 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: archiving obsolete branches with a dated timestamp (20260709), directly matching the file created (BRANCH_ARCHIVE_20260709.md).
Description check ✅ Passed The PR description matches the template structure but lacks substantive content in the Summary and Notes sections, both empty (only dashes). However, the verification and governance checklists are present with unchecked items, meeting the structural requirements despite minimal explanation of what is being archived or why.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Generated And Sensitive Files ✅ Passed Only changed file is a markdown archive doc; no secrets, envs, caches, build outputs, or other sensitive/generated artifacts found.
Verification Claims ✅ Passed No unsupported verification-passed claims found; the PR description is empty aside from unchecked checklists, and the new archive note is descriptive only.
Risky Git Or Deployment Actions ✅ Passed The PR only adds an archive note using Git's ours strategy; it contains no force-push, reset-hard, destructive clean, or deployment instructions.
Supabase Project And Schema Safety ✅ Passed PR only adds a markdown branch archive; no app/script/migration/env changes or destructive Supabase schema/data updates are present.
Runtime And Package Manager Integrity ✅ Passed PASS: HEAD only adds BRANCH_ARCHIVE_20260709.md; npm lockfile, engine-strict, and Node 24/npm 11 pins are unchanged.
Api Route Failure Handling ✅ Passed Only BRANCH_ARCHIVE_20260709.md was added; no API, server-action, RAG, ingestion, or provider code changed.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/archive-obsolete-branches-20260709
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch codex/archive-obsolete-branches-20260709

Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 98e43883b2

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread BRANCH_ARCHIVE_20260709.md
@BigSimmo
BigSimmo merged commit 0cef103 into main Jul 9, 2026
11 of 12 checks passed
@BigSimmo
BigSimmo deleted the codex/archive-obsolete-branches-20260709 branch July 9, 2026 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants